Skip to content

docs: rate limits table with search#2460

Draft
unatasha8 wants to merge 11 commits intomasterfrom
feat/rate-limits-table-with-search
Draft

docs: rate limits table with search#2460
unatasha8 wants to merge 11 commits intomasterfrom
feat/rate-limits-table-with-search

Conversation

@unatasha8
Copy link
Contributor

  • I have read the contributing guidelines and signed the CLA.
  • I have referenced an issue containing the design document if my change introduces a new feature.
  • I have read the security policy.
  • I confirm that this pull request does not address a security vulnerability.
    If this pull request addresses a security vulnerability,
    I confirm that I got approval (please contact security@ory.com) from the maintainers to push the changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added the necessary documentation within the code base (if appropriate).

Further comments

@unatasha8 unatasha8 changed the title docs: Rate limits table with search docs: rate limits table with search Mar 5, 2026

- **Burst limit**: Maximum requests per second (rps), allowing for short traffic spikes.
- **Sustained limit**: Maximum requests per minute (rpm), ensuring consistent performance over time.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we split the technical below into another document? It feels odd to get into the "how" when we are explaining the why above.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think if we split it, they won't see/read it... the above is 'what' project rate limits are and below is 'how' they need to implement them

Update bucket naming 'threshold' definitions to match new definitions
@tricky42
Copy link
Contributor

tricky42 commented Mar 11, 2026

Hey Una,

Here's the banner text, along with a proposal for a small restructuring that I think makes the overall navigation cleaner.

Proposed structure

Three pages:

  • /rate-limits — new parent/overview page (already drafted, can share the file)
  • /rate-limits-legacy — the current /rate-limits content, unchanged
  • /rate-limits-new — stays as-is

The key benefit: since /rate-limits becomes the overview page, every customer who has that URL bookmarked will automatically land there and see the migration notice — before navigating to whichever detail page applies to them. No extra promotion needed; the restructuring does the communication work for us.

New visitors coming via search will also land on the overview first, which means everyone gets context on old vs. new before diving into the details.

The banners on the detail pages are then a secondary safety net for anyone who arrives there directly.

Banner text for /rate-limits-legacy:

This page describes the legacy rate limiting system, which applies to existing Ory Network customers who haven't been migrated yet. If you're a new customer or have already been migrated, see the new rate limits. Learn about both systems and the migration plan → /docs/guides/rate-limits

Banner text for /rate-limits-new:

This page describes the new rate limiting system, which applies to all new Ory Network customers and existing customers after migration. If you're an existing customer and haven't received a migration notice yet, see the legacy rate limits. Learn about both systems and the migration plan → /docs/guides/rate-limits

Both banners: info severity.

One open item on the parent page: I included a note that customers can check which system applies to them in the Ory Console under Settings → Rate Limits. That's just an idea for now — would actually be quite simple to implement and a nice touch, but it's not there yet. I've marked it as a placeholder so we can either build it or drop the reference before publishing.

Let me know if the restructuring works for you!

rate-limits.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants